Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retry tests 100 #4

Closed
wants to merge 22 commits into from
Closed

Retry tests 100 #4

wants to merge 22 commits into from

Conversation

CharlieTLe
Copy link
Owner

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

CharlieTLe and others added 22 commits June 2, 2024 18:31
* Create ValidateMetrics

The validation package is exporting Prometheus counters that are using
the default Prometheus registry. This is problematic when unit tests are
running tests in parallel because the distributors and ingesters will
use the exported counters in their tests.

Signed-off-by: Charlie Le <[email protected]>

* Address linting errors

Signed-off-by: Charlie Le <[email protected]>

* Ignore AlreadyRegisterErr when registering ValidateMetrics

These metrics could be registered by the distributor and the ingester in
single binary mode.

Signed-off-by: Charlie Le <[email protected]>

* Set registry when creating ingesters

Signed-off-by: Charlie Le <[email protected]>

---------

Signed-off-by: Charlie Le <[email protected]>
The tests rely on the CPU's wall clock which can be unreliable if there is CPU starvation. If we give the tests enough time to run, they are more much more reliable and resilient to CPU starvation.

Signed-off-by: Charlie Le <[email protected]>
* Genetic per labelset limit

Signed-off-by: alanprot <[email protected]>

* Error if duplicate labelset

Signed-off-by: alanprot <[email protected]>

* Test limiter

Signed-off-by: alanprot <[email protected]>

* Changelog + doc

Signed-off-by: alanprot <[email protected]>

* lint

Signed-off-by: alanprot <[email protected]>

* avoid calling labelset.String() multiples times

Signed-off-by: alanprot <[email protected]>

---------

Signed-off-by: alanprot <[email protected]>
* s3: no need to retry and wait when last error is context cancel or deadline exceeded

Signed-off-by: Ben Ye <[email protected]>

* add tests

Signed-off-by: Ben Ye <[email protected]>

* lint

Signed-off-by: Ben Ye <[email protected]>

---------

Signed-off-by: Ben Ye <[email protected]>
…t#6007)

* Using backfillProcessor to store items on cache

Signed-off-by: alanprot <[email protected]>

* using thanos CacheType strings

Signed-off-by: alanprot <[email protected]>

* rafactor backfillDroppeItems metrics

Signed-off-by: alanprot <[email protected]>

* Creating new cortex_store_multilevel_index_cache_store_dropped_items_total metric

Signed-off-by: alanprot <[email protected]>

---------

Signed-off-by: alanprot <[email protected]>
* support ingesting native histograms

Signed-off-by: Ben Ye <[email protected]>

* fix lint

Signed-off-by: Ben Ye <[email protected]>

* update doc to mark experimental feature

Signed-off-by: Ben Ye <[email protected]>

* update changelog and doc

Signed-off-by: Ben Ye <[email protected]>

* update test

Signed-off-by: Ben Ye <[email protected]>

* fix test

Signed-off-by: Ben Ye <[email protected]>

* address comment; handle histogram partial append errors

Signed-off-by: Ben Ye <[email protected]>

---------

Signed-off-by: Ben Ye <[email protected]>
…#6004)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.53.16 to 1.53.20.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](aws/aws-sdk-go@v1.53.16...v1.53.20)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Upgrade thanos

Signed-off-by: Justin Jung <[email protected]>

* Fix build

Signed-off-by: Justin Jung <[email protected]>

---------

Signed-off-by: Justin Jung <[email protected]>
…exproject#6024)

Bumps [github.com/VictoriaMetrics/fastcache](https://github.com/VictoriaMetrics/fastcache) from 1.12.1 to 1.12.2.
- [Commits](VictoriaMetrics/fastcache@v1.12.1...v1.12.2)

---
updated-dependencies:
- dependency-name: github.com/VictoriaMetrics/fastcache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* update prometheus and thanos version to latest main

Signed-off-by: Ben Ye <[email protected]>

* lint

Signed-off-by: Ben Ye <[email protected]>

* unpin prometheus common

Signed-off-by: Ben Ye <[email protected]>

---------

Signed-off-by: Ben Ye <[email protected]>
…t#6023)

Bumps [github.com/minio/minio-go/v7](https://github.com/minio/minio-go) from 7.0.70 to 7.0.71.
- [Release notes](https://github.com/minio/minio-go/releases)
- [Commits](minio/minio-go@v7.0.70...v7.0.71)

---
updated-dependencies:
- dependency-name: github.com/minio/minio-go/v7
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…6009)

Bumps [github.com/Azure/azure-sdk-for-go/sdk/azidentity](https://github.com/Azure/azure-sdk-for-go) from 1.5.2 to 1.6.0.
- [Release notes](https://github.com/Azure/azure-sdk-for-go/releases)
- [Changelog](https://github.com/Azure/azure-sdk-for-go/blob/main/documentation/release.md)
- [Commits](Azure/azure-sdk-for-go@sdk/internal/v1.5.2...sdk/azcore/v1.6.0)

---
updated-dependencies:
- dependency-name: github.com/Azure/azure-sdk-for-go/sdk/azidentity
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Build image in github actions

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Bug fixes

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Target master

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Fix save-multiarch-build-image

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Include QEMU and buildx action

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Let's simplify and tests first

Signed-off-by: Friedrich Gonzalez <[email protected]>

* test push

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Push intermediate images

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Re-enable full build, update docs and make sure push is only possible from master

Signed-off-by: Friedrich Gonzalez <[email protected]>

* Fetch tags and use Makefile in build-image

Signed-off-by: Friedrich Gonzalez <[email protected]>

---------

Signed-off-by: Friedrich Gonzalez <[email protected]>
v2 is deprecated and will not work after June 30

Signed-off-by: Friedrich Gonzalez <[email protected]>
Signed-off-by: Charlie Le <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants